[SYCL] Common Reference Semantics for accessor class #63
Conversation
Signed-off-by: Vyacheslav N Klochkov <vyacheslav.n.klochkov@intel.com>
| // ... first descriptor (translated to pointer kernel parameter): | ||
| // The parameter is a SYCL accessor object. | ||
| // The Info field of the parameter descriptor for accessor contains | ||
| // two template parameters packed into thid integer field: |
There was a problem hiding this comment.
I do not understand this sentence.
But great to have all this new documentation. I love reading a lot of comments while reading some code. :-)
By the way @vladimirlaz can you wait a little bit longer before merging the commits? Because of the different timezones we do not have time to just look at the commits before merging...
One more day should be enough.
There was a problem hiding this comment.
Agree to wait till next working day to let everybody provide comments
There was a problem hiding this comment.
I tried to make the comment better than it was before (it though had some misprint at L714: thid -> this).
It is sad that it causes questions and needs some rephrasing/rewriting.
Would this one be better? (I could update this place sometime in future when fix this file again).
The 2nd argument of the call H.addParamDesc() below defines what is written
to the accessor parameter descriptor and its int field 'Info'.
For the template class accessor<Type, Dimensions, Mode, Target, IsPlaceHolder>
it keeps both 'Target' and 'Dimensions' packed into that int field 'Info' as:
(Target | (Dimensions << 11)).
We are about to add enum attributes with AttrKind numbers >= 63. This means we cannot use AttrKind #63 to test for an invalid attribute number in the RAW format anymore. This patch changes the number of an invalid attribute to #255. There is no change to the character of the tests. Differential Revision: https://reviews.llvm.org/D64531 llvm-svn: 365722
CONFLICT (content): Merge conflict in clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp
CONFLICT (content): Merge conflict in clang/lib/Driver/ToolChains/Clang.cpp
CONFLICT (content): Merge conflict in clang/lib/Sema/SemaLookup.cpp
3 participants
Signed-off-by: Vyacheslav N Klochkov vyacheslav.n.klochkov@intel.com